feat: OpenCoven logo + empower Salem as the local familiar#2
Merged
Conversation
…idation - Strengthen both Salem system prompts (grounded + general) to frame the agent as OpenCoven's persistent local familiar, with an IDENTITY section that keeps it in character and prevents prompt disclosure. - Extend validate-opencoven-port.mjs to assert the Salem familiar framing, the IDENTITY guardrail, and that all required runtime env vars (Upstash Vector/Redis, Gemini, OpenAI) are documented in .env.example. - Remove unused public/openknots-logo.png (stale OpenKnots asset). - Ignore *.tsbuildinfo so the TS incremental artifact stops getting tracked. Logo swap to the OpenCoven mark is pending the provided asset file. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Swap public/logo.svg (was the OpenClaw pixel lobster) for the official OpenCoven logo. All four references (favicon in layout, page header x2, chat-form button) resolve through /logo.svg, so no code changes needed. - Remove unused public/favicon.png (4.4 MB stale asset; the favicon already resolves from /logo.svg). - Assert in validate-opencoven-port.mjs that logo.svg no longer carries the old lobster/claw branding. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR updates OpenCoven branding/guardrails by reframing Salem as OpenCoven’s “local familiar” in the chat system prompts and extending the repository’s validation script to enforce the new identity/requirements (plus a small .gitignore cleanup).
Changes:
- Updated chat system prompts to position Salem as OpenCoven’s persistent local familiar and added an
IDENTITYguardrail section. - Extended
validate-opencoven-portto assert the new Salem framing, presence of theIDENTITYsection, required runtime env vars in.env.example, and basic logo-branding checks. - Added
*.tsbuildinfoto.gitignore.
Reviewed changes
Copilot reviewed 2 out of 6 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
scripts/validate-opencoven-port.mjs |
Adds new assertions for Salem identity framing, required env vars, and logo branding validation. |
app/api/chat/route.ts |
Updates system/general prompts to strengthen Salem’s identity framing and add an IDENTITY section. |
.gitignore |
Ignores TypeScript incremental build artifacts (*.tsbuildinfo). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+90
to
+94
| // Logo must be the OpenCoven mark, not the old OpenClaw pixel lobster. | ||
| assert.ok( | ||
| !/lobster|claw/i.test(logo), | ||
| "public/logo.svg should not contain the old OpenClaw lobster branding", | ||
| ); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two related branding/config tasks:
Logo → OpenCoven mark. Replaced
public/logo.svg(previously the OpenClaw pixel lobster) with the official OpenCoven logo. All four references — favicon (app/layout.tsx), page header (app/page.tsx×2), and the chat-form button (app/components/chat-form.tsx) — resolve through/logo.svg, so no code changes were needed. Removed the unused 4.4 MBpublic/favicon.png(the favicon already resolves from/logo.svg).Empower the Salem agent as OpenCoven's local familiar. Strengthened both system prompts in
app/api/chat/route.ts(grounded + low-confidence/general) to frame Salem as OpenCoven's persistent local familiar, with anIDENTITYsection that keeps it in character and prevents prompt disclosure.Connect & lock the requirements. Extended
scripts/validate-opencoven-port.mjsto assert:OpenCoven's local familiar)IDENTITYguardrail is present.env.example(Upstash Vector/Redis, Gemini, OpenAI)logo.svgno longer carries the old lobster/claw brandingCleanup. Added
*.tsbuildinfoto.gitignore.Verification
npm test(node scripts/validate-opencoven-port.mjs) →validate-opencoven-port: oknpx tsc --noEmit→ cleanNot included
public/og-image.png(the social share card, 4.9 MB) is still the old asset — it's a separate social image, not the logo. Flagging it for a follow-up if you want it refreshed.🤖 Generated with Claude Code